Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update hosting-on-aws-amplify.md #2781

Closed
wants to merge 1 commit into from
Closed

Conversation

dallmo
Copy link

@dallmo dallmo commented Nov 29, 2024

in the section Hosting, added instructions on how to modify amplify.yml - because i just tried myself and Amplify does failed to detect, and i have to manually adjust the build settings like these proposed content changes.

in the section `Hosting`, added instructions on how to modify `amplify.yml` - because i just tried myself and Amplify does failed to detect, and i have to manually adjust the build settings like these proposed content changes.
@jmooring
Copy link
Member

add a file named amplify.yml to override the build settings in Amplify ;

Below this sentence you should show the content of a working file, not the default. Users want to cut/paste, and the default file is a dead-end.

@dallmo
Copy link
Author

dallmo commented Nov 30, 2024

sorry i missed that "/public" in the default file image attachment, which is missing in my AWS Amplify config so i thought my addition in line 47 can make a difference.

if you consider this proposed change is redundant, i will close this request to avoid confusions.

jmooring added a commit to jmooring/hugo-docs that referenced this pull request Dec 3, 2024
jmooring added a commit to jmooring/hugo-docs that referenced this pull request Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants